Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix:修复tcc-fence模式空回滚无限重试以及无法解决悬挂bug #684

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

yangwenbinch
Copy link

What this PR does:
解决tcc-fence无限回滚与无法解决悬挂问题
具体问题见链接:apache/incubator-seata-go-samples#53
Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:
此问题使tcc-fence模式下使用问题较大
Does this PR introduce a user-facing change?:

NONE

@yangwenbinch
Copy link
Author

Which issue(s) this PR fixes:
#685

Copy link

@luky116 luky116 removed the coding label Nov 23, 2024
}
log.Infof("Insert tcc fence suspend record xid: %s, branchId: %d", xid, branchId)
return nil
return true, tx.Commit()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里是否要执行 commit?

@luky116
Copy link
Contributor

luky116 commented Dec 21, 2024

正在测试

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants